GtkColorPlane: Actually add the long press gesture
authorMatthias Clasen <mclasen@redhat.com>
Fri, 2 May 2014 01:00:28 +0000 (21:00 -0400)
committerCarlos Garnacho <carlosg@gnome.org>
Fri, 23 May 2014 17:54:28 +0000 (19:54 +0200)
This was missed because gtk_widget_remove_controller silently
does nothing on unowned controllers.

gtk/gtkcolorplane.c

index 770cb8dc86abf33d12ff558cb876f11a17eeea7c..9a81af1df084351e27a31ea0f0633fd3cc8148bb 100644 (file)
@@ -427,6 +427,7 @@ gtk_color_plane_init (GtkColorPlane *plane)
   plane->priv->long_press_gesture = gtk_gesture_long_press_new (GTK_WIDGET (plane));
   g_signal_connect (plane->priv->long_press_gesture, "pressed",
                     G_CALLBACK (hold_action), plane);
+  gtk_widget_add_controller (GTK_WIDGET (plane), GTK_EVENT_CONTROLLER (plane->priv->long_press_gesture));
 }
 
 static void
@@ -441,8 +442,7 @@ plane_finalize (GObject *object)
   g_clear_object (&plane->priv->s_adj);
   g_clear_object (&plane->priv->v_adj);
 
-  gtk_widget_remove_controller (GTK_WIDGET (object),
-                                GTK_EVENT_CONTROLLER (plane->priv->long_press_gesture));
+  gtk_widget_remove_controller (GTK_WIDGET (object), GTK_EVENT_CONTROLLER (plane->priv->long_press_gesture));
   g_clear_object (&plane->priv->long_press_gesture);
 
   G_OBJECT_CLASS (gtk_color_plane_parent_class)->finalize (object);